From d7a4a39acb5f1f6b45724f6f14e01626fc6cbdf6 Mon Sep 17 00:00:00 2001 From: Keir Fraser Date: Fri, 18 Sep 2009 08:26:53 +0100 Subject: [PATCH] Revert 20194:582970a2d2dc Excessively slows down domain creation in debug builds. Signed-off-by: Keir Fraser --- xen/arch/x86/mm/p2m.c | 30 ------------------------------ 1 file changed, 30 deletions(-) diff --git a/xen/arch/x86/mm/p2m.c b/xen/arch/x86/mm/p2m.c index d0c868dce0..48132f3698 100644 --- a/xen/arch/x86/mm/p2m.c +++ b/xen/arch/x86/mm/p2m.c @@ -1473,36 +1473,6 @@ int set_p2m_entry(struct domain *d, unsigned long gfn, mfn_t mfn, 9 : 0; else order = 0; - -#ifndef NDEBUG - /* PoD code assumes that a page owned by the domain, not from the xenheap, and in the p2m - * is on the domain page list. Verify this assumption. */ - if ( mfn_valid(mfn) - && p2m_is_ram(p2mt) - && page_get_owner(mfn_to_page(mfn))==d - && ( (mfn_to_page(mfn)->count_info & PGC_xen_heap) == 0 ) ) - { - struct page_info *p, *q; - - p = mfn_to_page(mfn); - - spin_lock(&d->page_alloc_lock); - - /* Walk the domain page list and make sure this page is on it... */ - for ( q = page_list_first(&d->page_list) ; q; q = page_list_next(q, &d->page_list) ) - if ( q == p ) - break; - if ( !q ) - { - printk("%s: mfn %lx owned by d%d, not xen_heap, but not on domain page_list!\n", - __func__, mfn_x(mfn), d->domain_id); - BUG(); - } - - spin_unlock(&d->page_alloc_lock); - } -#endif - rc = d->arch.p2m->set_entry(d, gfn, mfn, order, p2mt); gfn += 1ul << order; if ( mfn_x(mfn) != INVALID_MFN ) -- 2.30.2